Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop Payee Check Constraint #3287

Merged
merged 9 commits into from
Jun 27, 2023
Merged

Drop Payee Check Constraint #3287

merged 9 commits into from
Jun 27, 2023

Conversation

dfilteau
Copy link
Contributor

@dfilteau dfilteau commented Jun 23, 2023

PIMS_ACQUISITION_PAYEE table's ACQPAY_COMP_REQ_PAYEE_TCC check constraint has multiple, parentheses-separated conditions to ensure that there are NONE or ONE payee. Unfortunately, SQL Server strips out all parentheses and then prioritizes ANDs over ORs in the check constraint, resulting in unpredictable results.

The check constraint is being dropped until a better solution is found.

dfilteau and others added 6 commits June 15, 2023 12:04
Set all null lease payment statuses to their correct value based on the lease  term's payment amount + GST
Set all null lease payment statuses to their correct value based on the lease  term's payment amount + GST
@github-actions
Copy link
Contributor

✅ No secrets were detected in the code.

@github-actions
Copy link
Contributor

✅ No secrets were detected in the code.

@FuriousLlama FuriousLlama self-assigned this Jun 27, 2023
@FuriousLlama FuriousLlama added bug Something isn't working db-migration labels Jun 27, 2023
@github-actions
Copy link
Contributor

✅ No secrets were detected in the code.

@github-actions
Copy link
Contributor

✅ No secrets were detected in the code.

@FuriousLlama FuriousLlama merged commit 01b86f8 into bcgov:test Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working db-migration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants